Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

South Africa Class-1 - Jonathan Heugh - JavaScript-Core-2-Week3#56

Open
Jontas77 wants to merge 11 commits into
CodeYourFuture:mainfrom
Jontas77:main
Open

South Africa Class-1 - Jonathan Heugh - JavaScript-Core-2-Week3#56
Jontas77 wants to merge 11 commits into
CodeYourFuture:mainfrom
Jontas77:main

Conversation

@Jontas77
Copy link
Copy Markdown

@Jontas77 Jontas77 commented Mar 24, 2021

Your Details

  • Your Name: Jonathan
  • Your City: Cape Town
  • Your Slack Name: Jonathan

Homework Details

  • Module: JavaScript Core2
  • Week: 3

@bradleymubenga777
Copy link
Copy Markdown

Hey Bro, I noticed that you forgot to include the homework details. Please do so.

@Jontas77
Copy link
Copy Markdown
Author

Hey Bro, I noticed that you forgot to include the homework details. Please do so.

Sure bro

Copy link
Copy Markdown

@ezekiel444 ezekiel444 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clapping... Impressive Johnathan. But:

  1. ALARM CLOCK without inputting any value and pressing the start button. your code start runing in the background. To check inspect your page from the devtools to see.
    input a negative value first then input a positive value and see how your code behave
    pause alarm clock not working.
  2. SLIDE SHOW first time press the back/forward button multiple times and view your console for not found error.
    according to the requirement the slide is not showing the current index of the displayed image.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants